Skip to content

fix: remove KOTS template functions from helmchart.yaml values#29

Merged
aa-replicated merged 2 commits into
mainfrom
fix/helmchart-remove-kots-values
Apr 12, 2026
Merged

fix: remove KOTS template functions from helmchart.yaml values#29
aa-replicated merged 2 commits into
mainfrom
fix/helmchart-remove-kots-values

Conversation

@aa-replicated
Copy link
Copy Markdown
Owner

Summary

  • Removes {{repl ConfigOption "admin_secret"}}, {{repl ConfigOption "site_name"}}, and {{repl ImagePullSecretName }} template function calls from helmchart.yaml
  • Keeps image.tag: 'repl{{ .VersionLabel }}' (Go-template style, not a KOTS function call)
  • EC only needs name/chartVersion to match the chart archive; value injection happens in the KOTS Admin Console flow

Why

EC v3 parses helmchart.yaml with a standard YAML parser before KOTS template functions are resolved. The {{repl FunctionName }} syntax made the file unparseable at that stage, causing:

unmarshal file to chart helmchart.yaml: yaml: line 9: did not find expected key

Test plan

  • Merge and let CI publish a release
  • Attempt EC v3 upgrade — the did not find expected key error should be gone

aa-replicated and others added 2 commits April 11, 2026 21:43
EC v3 parses helmchart.yaml with a strict YAML parser before KOTS
template functions are resolved, causing 'did not find expected key'
on upgrade. Remove ConfigOption and ImagePullSecretName template
calls — EC only needs name/chartVersion to locate the chart archive.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
- Switch from {{repl FunctionName "arg"}} to repl{{ FunctionName `arg`}}
  syntax per Replicated docs
- Remove global.imagePullSecrets (not needed for ghcr.io images)
- Add builder section required for air gap support

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
@aa-replicated aa-replicated merged commit 29e6da2 into main Apr 12, 2026
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant